Skip to content

TST: GH30999 Add match=msg to test_nat_comparisons_invalid #38564

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 18, 2020

Conversation

moink
Copy link
Member

@moink moink commented Dec 18, 2020

This pull request xref #30999 to remove bare pytest.raises. It doesn't close that issue as I have only addressed one file: pandas/tests/scalar/test_nat.py . In that file there was only one test that had a bare pytest.raises and I added a message to the two instances of pytest.raises in that test. It required modifications to the test parameters in order to populate the message.

I did not add a whatsnew entry since it's only a tiny change to one test. Let me know if I should add one (and I am a bit unclear on how, i.e. what version this would end up in).

This is my first ever pull request to an open source project. I am expecting that I will have to make changes before it's accepted; thanks for your patience.

@jreback jreback added the Missing-data np.nan, pd.NaT, pd.NA, dropna, isnull, interpolate label Dec 18, 2020
@jreback jreback added this to the 1.3 milestone Dec 18, 2020
@jreback jreback merged commit e68016c into pandas-dev:master Dec 18, 2020
@jreback
Copy link
Contributor

jreback commented Dec 18, 2020

thanks @moink

@moink moink deleted the add_match_msg_to_pytest_raises_nat branch December 21, 2020 12:37
luckyvs1 pushed a commit to luckyvs1/pandas that referenced this pull request Jan 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Missing-data np.nan, pd.NaT, pd.NA, dropna, isnull, interpolate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants